Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Python updates #1392

Merged
merged 20 commits into from
Sep 1, 2016
Merged

Python updates #1392

merged 20 commits into from
Sep 1, 2016

Conversation

annatisch
Copy link
Member

Fix for issue #1356 and #1300

@yugangw-msft
Copy link
Contributor

this fixes Azure/azure-cli#754

@tbombach
Copy link
Member

tbombach commented Sep 1, 2016

Looks good to me. Let me know if I should wait to merge if you want other Python eyes on this

@lmazuel
Copy link
Member

lmazuel commented Sep 1, 2016

I'm happy and my tests are ok! @tbombach for me you can merge. :)

@tbombach tbombach merged commit a672edd into Azure:master Sep 1, 2016
@tbombach
Copy link
Member

tbombach commented Sep 1, 2016

Merged

@yugangw-msft
Copy link
Contributor

@tbombach thanks!
@annatisch @lmazuel , do we need to up the version, publish and let sdk-for-python to pick it up?

@lmazuel
Copy link
Member

lmazuel commented Sep 1, 2016

@yugangw-msft will be msrest/msrestazure 0.4.x versions, no need to republish the SDK for that. I plan to to do it today.

@yugangw-msft
Copy link
Contributor

@lmazuel thanks!

@lmazuel
Copy link
Member

lmazuel commented Sep 1, 2016

@yugangw-msft Released, see #1398

tbombach pushed a commit to tbombach/autorest that referenced this pull request Sep 2, 2016
* Added default value to doc strings

* Added support for default date objects

* Fix for global parameter quoting

* Fixed reference

* Regenerated Tests

* Better handling of constants in operations

* Fixed doc string typo

* Added default value to doc strings

* Added support for default date objects

* Removed extra doc defaults

* Removed extra doc defaults

* Some cleanups

* Some cleanups

* Fix for None object serialization

* Removed parsing of Namespace

* LRO exception fixes

* Added check for invalid namespace

* Moved namespace validation

* Regenerated tests

* Reverted duplicate fix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants